Skip to content

fix(a2a): stabilize smoke tests with Redis - #45

Merged
openjiuwenai merged 2 commits into
openJiuwen-ai:developfrom
openjiuwenai:sync/pr-133
Jul 30, 2026
Merged

fix(a2a): stabilize smoke tests with Redis#45
openjiuwenai merged 2 commits into
openJiuwen-ai:developfrom
openjiuwenai:sync/pr-133

Conversation

@openjiuwen-sync-bot

@openjiuwen-sync-bot openjiuwen-sync-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown

Paired: GitHub #45GitCode !133

变更说明

  • 放宽 Agent D 流式和非流式恢复场景对最终 LLM 展示文本的断言,保留任务状态、taskId、路由及最终节点等稳定协议和流程检查。
  • 修正 Redis TaskStore 慢写场景下的节流时间点,按实际持久化完成时间计算下一次写入窗口,避免 streaming artifact 逐条同步写 Redis。
  • INPUT_REQUIRED 队列只有在事件排空后才关闭;若超时则保持打开,避免迟到的中断状态因关闭竞态而无法交付。
  • 增加慢速 delegate 写入和队列排空超时的回归测试。

问题原因

Agent D 的结构化结果经过 Agent B 和 Agent A 的 ReAct 层后可能被模型重新表述,展示格式不是稳定协议,严格文本断言会造成随机失败。

Redis 场景中,原节流逻辑在慢速 delegate.save() 前记录时间。单次 Redis 保存超过节流窗口后,下一事件会再次同步持久化,导致事件队列积压。积压未在 60 秒内排空时,INPUT_REQUIRED 的消费队列仍被关闭,远端最终被判定为 REMOTE_UNAVAILABLE

验证

  • 聚焦回归测试:21 个测试通过,0 failure,0 error。
  • mvn -pl agent-service-app -am testagent-service-app 385 个测试通过,0 failure,0 error。
  • 本次 4 个 Java 文件 formatter 定向校验与 Checkstyle 通过。
  • 使用 application-a2a-redis.local.ymlcheckpointer.type=redis)完整执行 smoke-a2a.sh:计算器、Agent C 流式/非流式、Agent D 流式/非流式中断恢复全部通过,包括原失败点 [6b]
  • git diff --check 通过;测试结束后端口 18090-18093 均已释放。

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


xiaoming-2026 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@openjiuwen-collaboration-bot

Copy link
Copy Markdown

head_sha: 2d50a60e20d9a0cdd5d4671def32fa5aa9c06fc7

TASK STATUS DETAILS
CodeCheck ✅SUCCESS Click here
AntiPoison ✅SUCCESS Click here
Software Composition Analysis ✅SUCCESS Click here
Maven Build ✅SUCCESS See CHECK tab

@openjiuwen-sync-bot openjiuwen-sync-bot Bot changed the title test(a2a): stabilize workflow smoke assertions fix(a2a): stabilize smoke tests with Redis Jul 29, 2026
@openjiuwen-collaboration-bot

Copy link
Copy Markdown

head_sha: 681b9a01fa5cd52e0ccee774c3e7a4962f0393e5

TASK STATUS DETAILS
CodeCheck ✅SUCCESS Click here
AntiPoison ✅SUCCESS Click here
Software Composition Analysis ✅SUCCESS Click here
Maven Build ✅SUCCESS See CHECK tab

@openjiuwenai
openjiuwenai merged commit 40be367 into openJiuwen-ai:develop Jul 30, 2026
1 check was pending
@openjiuwen-release-bot

Copy link
Copy Markdown

🤖 This pull request has been merged, and the target branch is up to date.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants